-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
View claim #300
View claim #300
Conversation
I don't understand |
lol it was a joke, I just added a 3 second promise before showing the success to better simulate the real thing for the sake of QA 😂 |
Doesn't work |
We do indeed get the hash and it doesn't change from it being sent and it being confirmed afaik. So I think either Blockscan doesn't have it yet and returns a 404 or there is some other issue, likely it hadn't populated yet. Idk if you'll agree but explorers tend to hold the tx until it is confirmed, perhaps Blockscan is different. I will think more on your comment and include this attempted optimization (properly implemented if it's doable) alongside any others. I'll aim to have a PR open for this over the weekend. Was it the preview deployment that you tested I assume? Would you be able to share the PK for the wallet used for the preview deployments with me? |
It was merged to production and I claimed and tested. You should run locally and claim/test |
I added a little delay effect, no autographs please
claim-with-wait.mp4